Skip to content

fix repeated content address example#524

Merged
jonboulle merged 1 commit intoopencontainers:masterfrom
xiekeyang:cas-example
Jan 25, 2017
Merged

fix repeated content address example#524
jonboulle merged 1 commit intoopencontainers:masterfrom
xiekeyang:cas-example

Conversation

@xiekeyang
Copy link
Contributor

@xiekeyang xiekeyang commented Jan 24, 2017

should not be present 2 identicial cas file under a dir.

Signed-off-by: xiekeyang xiekeyang@huawei.com

image-layout.md Outdated
$ cat ./blobs/sha256/9b97579de92b1c195b85bb42a11011378ee549b02d7fe9c17bf2a6b35d5cb079 | jq
{
"schemaVersion": 2,
"mediaType": "application/vnd.oci.image.manifest.list.v1+json",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, this was removed in #411

@xiekeyang
Copy link
Contributor Author

@jonboulle Oh, it is my fault. Updated it, PTAL.

Signed-off-by: xiekeyang <xiekeyang@huawei.com>
Copy link
Contributor

@jonboulle jonboulle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vbatts
Copy link
Member

vbatts commented Jan 25, 2017

LGTM

Approved with PullApprove

@vbatts
Copy link
Member

vbatts commented Jan 25, 2017

... why did the other approval fall away?

@jonboulle
Copy link
Contributor

Lgtm

@jonboulle
Copy link
Contributor

jonboulle commented Jan 25, 2017

LGTM

Approved with PullApprove

@jonboulle jonboulle merged commit a31a6e1 into opencontainers:master Jan 25, 2017
@xiekeyang xiekeyang deleted the cas-example branch January 26, 2017 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants